Conversation
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
c9f746a to
58c0d08
Compare
…ove webhookUrl, clarify usedBYOKKey
|
/bonk review this |
|
1 issue needs fixing, 2 suggestions for improvement. CRITICAL
MEDIUM
LOW
Would you like me to fix the |
|
/bonk review |
|
Based on my review, the file is structurally sound. The No issues found. The documentation follows the style guide:
Note: The PR title indicates "[WIP do not review]". The previous automated review incorrectly flagged the TypeScriptExample as missing when it is in fact properly implemented. |
|
/bonk review this |
|
Based on my review of PR #29345, here are my findings: Review SummaryFiles reviewed:
✅ What's GoodDocumentation Page (
Changelog:
📝 Suggestions1. Add TypeScript tag ( tags:
- AI
- TypeScript2. Changelog date ( ✅ ApprovalThe documentation is well-structured and follows the style guide. The Run API documentation is clear, with good examples in both curl and Workers AI binding formats. No blocking issues found. |
| - **BYOK (Gateway Key Store)** — Store your provider API keys with Cloudflare. Refer to [BYOK](/ai-gateway/configuration/bring-your-own-keys/). | ||
|
|
||
| :::caution | ||
| Passing provider API keys directly in request headers is not supported with the Run API. You must use Unified Billing or BYOK (Gateway Key Store) to authenticate with upstream providers. |
There was a problem hiding this comment.
"AI Gateway secrets store" maybe? Gateway Key Store we have not used before as a phrase
|
|
||
| | Model | Type | Provider | | ||
| | --- | --- | --- | | ||
| | [`google/nano-banana`](/workers-ai/models/google/nano-banana/) | Image generation | Google Vertex AI | |
There was a problem hiding this comment.
does this link really work? i'm not seeing it in this PR
There was a problem hiding this comment.
ah it's in Charlie's PR -- will make sure that gets merged first
Summary
Screenshots (optional)
Documentation checklist